Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Trimmed set of translations built by default. #2649

Merged

Conversation

petterreinholdtsen
Copy link
Collaborator

@petterreinholdtsen petterreinholdtsen commented Sep 20, 2023

Only translations touched the last 12 months or where >= 2% of the strings
are translated according to Weblate get build rules in docs/src/Submakefile, and
only languages with >50% get Debian packages with the translations. The new
guidelines are documented in docs/README.adoc.

The set of translations built and published on https://linuxcnc.org/docs/devel/html/
are now Arabic, German, Spanish, French, Norwegian Bokmål, Russian, Swedish, Turkish
and Simplified Chinese. Hungarian and Vietnamese are no longer built. Reduced the
list in index.html to the built ones.

Debian packages for French, Spanish and Simplified Chinese are no longer created.

@SebKuzminsky
Copy link
Collaborator

This is a good improvement and we should merge it.

@smoe
Copy link
Contributor

smoe commented Sep 20, 2023

We need to do something about translations that have no active maintainers, indeed. And I agree that such a policy would be an improvement.

@petterreinholdtsen petterreinholdtsen force-pushed the docs-trim-translations branch 2 times, most recently from d72eb11 to 401f18c Compare September 20, 2023 21:37
@petterreinholdtsen
Copy link
Collaborator Author

petterreinholdtsen commented Sep 20, 2023 via email

@petterreinholdtsen petterreinholdtsen force-pushed the docs-trim-translations branch 3 times, most recently from 7a551d3 to 46f2858 Compare September 21, 2023 06:36
Only translations touched the last 12 months or where >= 2% of the strings
are translated according to Weblate get build rules in docs/src/Submakefile, and
only languages with >50% get Debian packages with the translations.  The new
guidelines are documented in docs/README.adoc.

The set of translations built and published on https://linuxcnc.org/docs/devel/html/
are now Arabic, German, Spanish, French, Norwegian Bokmål, Russian, Swedish, Turkish
and Simplified Chinese.  Hungarian and Vietnamese are no longer built.  Reduced the
list in index.html to the built ones.

Debian packages for French, Spanish and Simplified Chinese are no longer created.
@hansu
Copy link
Member

hansu commented Sep 21, 2023

This is a good improvement and we should merge it.

I agree. So any objections on the details?

@smoe
Copy link
Contributor

smoe commented Sep 21, 2023

I frankly came to the conclusion that I prefer anyone talented enough to speak two languages and have a technical understanding of what is going on to please contribute to the English documentation. The translation we should address to see automated.

@petterreinholdtsen
Copy link
Collaborator Author

[Steffen Möller]

I frankly came to the conclusion that I prefer anyone talented enough to speak two languages and have a technical understand of what is going on to please contribute to the English documentation. The translation we should address to see automated.

While that might work for German, there are other languages where automated translation is not doing a good job. I find the automated Norwegian Bokmål translations seen so far to be of too low quality. So I believe we need to help translators also moving forward.

@SebKuzminsky SebKuzminsky merged commit eda2907 into LinuxCNC:master Sep 22, 2023
11 checks passed
@petterreinholdtsen petterreinholdtsen deleted the docs-trim-translations branch October 7, 2023 13:54
@petterreinholdtsen petterreinholdtsen mentioned this pull request Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants